Skip to content

fix(session-ingest): extend queue retry delay#3709

Merged
johnnyeric merged 1 commit into
mainfrom
johnnyeric/session-ingest-queue-timeout
Jun 3, 2026
Merged

fix(session-ingest): extend queue retry delay#3709
johnnyeric merged 1 commit into
mainfrom
johnnyeric/session-ingest-queue-timeout

Conversation

@johnnyeric
Copy link
Copy Markdown
Contributor

Summary

  • Increase failed session-ingest queue retry delay from 60s to 5 minutes.
  • Define the retry delay as a named constant and update the queue consumer test.

Validation

  • pnpm --filter cloudflare-session-ingest exec vitest run src/queue-consumer.test.ts
  • pnpm run format:check

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Jun 3, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Clean two-file change that extracts the queue retry delay into a named constant (QUEUE_RETRY_DELAY_SECONDS = 5 * 60) and updates both the implementation and the test to use it, with no logic changes beyond the delay value increase from 60 s to 300 s.

Files Reviewed (2 files)
  • services/session-ingest/src/queue-consumer.ts
  • services/session-ingest/src/queue-consumer.test.ts

Fix these issues in Kilo Cloud


Reviewed by claude-4.6-sonnet-20260217 · 354,512 tokens

Review guidance: REVIEW.md from base branch main

@johnnyeric johnnyeric merged commit 26b63d6 into main Jun 3, 2026
16 checks passed
@johnnyeric johnnyeric deleted the johnnyeric/session-ingest-queue-timeout branch June 3, 2026 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants